Skip to content

Support custom conversions for the mapped Item fields#116

Merged
steleal merged 5 commits into
Restream:masterfrom
evgeniycheban:gh-115
May 22, 2025
Merged

Support custom conversions for the mapped Item fields#116
steleal merged 5 commits into
Restream:masterfrom
evgeniycheban:gh-115

Conversation

@evgeniycheban

Copy link
Copy Markdown
Member

Closes gh-115

@evgeniycheban evgeniycheban requested a review from steleal May 6, 2025 03:20
@evgeniycheban evgeniycheban self-assigned this May 6, 2025
@evgeniycheban evgeniycheban force-pushed the gh-115 branch 2 times, most recently from 8545f83 to 2b92cbd Compare May 9, 2025 02:04
@evgeniycheban evgeniycheban marked this pull request as draft May 11, 2025 16:18
@evgeniycheban

Copy link
Copy Markdown
Member Author

I changed it to draft to solve one design issue.

@evgeniycheban evgeniycheban marked this pull request as ready for review May 11, 2025 21:56
@evgeniycheban

Copy link
Copy Markdown
Member Author

I changed it to draft to solve one design issue.

It is ready for review.

@steleal

steleal commented May 12, 2025

Copy link
Copy Markdown
Contributor

Thanks

@evgeniycheban evgeniycheban force-pushed the gh-115 branch 5 times, most recently from fcb529d to 2475768 Compare May 12, 2025 19:27
@steleal

steleal commented May 13, 2025

Copy link
Copy Markdown
Contributor

Is the problem that PR solves relevant to the project? Is there compatibility with hibernate in the project roadmap?

@steleal

steleal commented May 13, 2025

Copy link
Copy Markdown
Contributor

It would be great to split one big commit into several atomic ones. It would be easier to review.

Comment thread src/main/java/ru/rt/restream/reindexer/convert/GenericFieldConverter.java Outdated
Comment thread src/main/java/ru/rt/restream/reindexer/convert/FieldConverterRegistryFactory.java Outdated
Comment thread src/main/java/ru/rt/restream/reindexer/convert/FieldConverterRegistryFactory.java Outdated
Comment thread src/main/java/ru/rt/restream/reindexer/ReindexerConfiguration.java
Comment thread src/main/java/ru/rt/restream/reindexer/convert/util/ConversionUtils.java Outdated
- Implemented resolution of convertible types

Closes Restreamgh-115
- Implemented FieldConverterRegistryFactory
- Exposed custom conversion api in ReindexerConfiguration

Closes Restreamgh-115
- Reconsidered internal mappings to use custom conversion api
- Implemented CollectionUtils to map various collection types

Closes Restreamgh-115
@evgeniycheban evgeniycheban changed the title Consider FieldConverter for mapped fields Support custom conversions for the mapped Item fields May 20, 2025
@evgeniycheban evgeniycheban requested a review from steleal May 20, 2025 00:26
@steleal steleal requested a review from MadSchemas May 22, 2025 05:43
@steleal steleal merged commit 492626f into Restream:master May 22, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support custom conversions for the mapped Item fields

3 participants